Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

config --xx don't need env_file being parsed #11025

Merged
merged 1 commit into from
Sep 21, 2023

Conversation

ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Sep 20, 2023

What I did
use compose-spec/compose-go#460

Related issue
closes #10995

(not mandatory) A picture of a cute animal, if possible in relation to what you did

@codecov
Copy link

codecov bot commented Sep 20, 2023

Codecov Report

Patch coverage: 37.50% and project coverage change: +0.03% 🎉

Comparison is base (52a641b) 57.39% compared to head (ab81db5) 57.43%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11025      +/-   ##
==========================================
+ Coverage   57.39%   57.43%   +0.03%     
==========================================
  Files         129      129              
  Lines       11229    11230       +1     
==========================================
+ Hits         6445     6450       +5     
+ Misses       4149     4147       -2     
+ Partials      635      633       -2     
Files Changed Coverage Δ
cmd/compose/config.go 34.35% <37.50%> (+0.40%) ⬆️

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ndeloof ndeloof requested review from a team, nicksieger, StefanScherer, ulyssessouza, glours, milas and laurazard and removed request for a team September 20, 2023 07:55
@ndeloof ndeloof marked this pull request as ready for review September 20, 2023 16:16
@ndeloof ndeloof requested a review from a team September 20, 2023 16:16
@ndeloof ndeloof force-pushed the SkipResolveEnvironment branch 2 times, most recently from 275130d to 4160bf2 Compare September 20, 2023 16:16
@glours glours force-pushed the SkipResolveEnvironment branch from 4160bf2 to 8cc39bf Compare September 21, 2023 08:07
go.mod Outdated
@@ -187,3 +187,5 @@ require (

// Override for e2e tests
replace github.com/cucumber/godog => github.com/laurazard/godog v0.0.0-20220922095256-4c4b17abdae7

replace github.com/compose-spec/compose-go => github.com/ndeloof/compose-go v1.2.4-0.20230920072217-f3748456c862
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

compose-go is up to date in the main branch, this replace rule can be removed

@ndeloof ndeloof force-pushed the SkipResolveEnvironment branch from 8cc39bf to 0dc3b43 Compare September 21, 2023 08:31
@ndeloof ndeloof force-pushed the SkipResolveEnvironment branch from 0dc3b43 to ab81db5 Compare September 21, 2023 08:47
@glours glours merged commit bd32ed1 into docker:main Sep 21, 2023
@ndeloof ndeloof deleted the SkipResolveEnvironment branch December 20, 2023 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] config --images should not fail on missing env_file
2 participants